libotutil: Plug a leak
authorKrzesimir Nowak <krzesimir@kinvolk.io>
Tue, 24 May 2016 08:36:38 +0000 (10:36 +0200)
committerAtomic Bot <atomic-devel@projectatomic.io>
Tue, 24 May 2016 11:36:15 +0000 (11:36 +0000)
There's no need to allocate the variant builder on a heap, so allocate
it on the stack and avoid a memory leak at the same time.

Closes: #307
Approved by: cgwalters

src/libotutil/ot-variant-utils.c

index 2dc07582c39065c4d29ab78058236199d592f55a..7b54542ee7ac1756400e9e1ac5d6696ad2ee259d 100644 (file)
 GVariant *
 ot_gvariant_new_empty_string_dict (void)
 {
-  return g_variant_builder_end (g_variant_builder_new (G_VARIANT_TYPE ("a{sv}")));
+  g_auto(GVariantBuilder) builder = {{0,}};
+
+  g_variant_builder_init (&builder, G_VARIANT_TYPE ("a{sv}"));
+  return g_variant_builder_end (&builder);
 }
 
 GVariant *